-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
chore(deps-dev): bump the jest-dependencies group with 2 updates #103412
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(deps-dev): bump the jest-dependencies group with 2 updates #103412
Conversation
41fc1fa to
4d5a4a5
Compare
| "jest": "30.2.0", | ||
| "jest-canvas-mock": "^2.5.2", | ||
| "jest-environment-jsdom": "30.0.4", | ||
| "jest-environment-jsdom": "30.2.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Jest Version Drift Causes Instability
The jest and jest-environment-jsdom packages were updated to 30.2.0, but babel-jest remains at 30.0.4 (line 201), creating a version mismatch within the Jest ecosystem. Jest packages from the same major version are designed to work together and should be kept in sync to avoid compatibility issues, test failures, or unexpected behavior during test execution.
4d5a4a5 to
542857e
Compare
542857e to
d3499b4
Compare
d3499b4 to
71781fa
Compare
71781fa to
3fe241b
Compare
3fe241b to
a06ac26
Compare
a06ac26 to
fea6d58
Compare
fea6d58 to
45aa78e
Compare
45aa78e to
242568f
Compare
242568f to
c612be6
Compare
c612be6 to
7e49a0f
Compare
219bc58 to
15cbe06
Compare
15cbe06 to
edfb8c9
Compare
edfb8c9 to
1a03946
Compare
5b83d5b to
823cf9b
Compare
823cf9b to
af29ae9
Compare
af29ae9 to
9012243
Compare
9012243 to
c8f342b
Compare
c8f342b to
1dfdc33
Compare
1dfdc33 to
dd0325a
Compare
dd0325a to
fe48f80
Compare
fe48f80 to
30aeb28
Compare
Bumps the jest-dependencies group with 2 updates: [jest](https://github.com/jestjs/jest/tree/HEAD/packages/jest) and [jest-environment-jsdom](https://github.com/jestjs/jest/tree/HEAD/packages/jest-environment-jsdom). Updates `jest` from 30.0.4 to 30.2.0 - [Release notes](https://github.com/jestjs/jest/releases) - [Changelog](https://github.com/jestjs/jest/blob/main/CHANGELOG.md) - [Commits](https://github.com/jestjs/jest/commits/v30.2.0/packages/jest) Updates `jest-environment-jsdom` from 30.0.4 to 30.2.0 - [Release notes](https://github.com/jestjs/jest/releases) - [Changelog](https://github.com/jestjs/jest/blob/main/CHANGELOG.md) - [Commits](https://github.com/jestjs/jest/commits/v30.2.0/packages/jest-environment-jsdom) --- updated-dependencies: - dependency-name: jest dependency-version: 30.2.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: jest-dependencies - dependency-name: jest-environment-jsdom dependency-version: 30.2.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: jest-dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
30aeb28 to
c04b089
Compare
| "jest": "30.2.0", | ||
| "jest-canvas-mock": "^2.5.2", | ||
| "jest-environment-jsdom": "30.0.4", | ||
| "jest-environment-jsdom": "30.2.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Directly reassigning window.location in the afterEach hook of api.spec.tsx will throw a TypeError because JSDOM v26, introduced via the jest update, makes this property read-only.
Severity: HIGH
Suggested Fix
In the afterEach hook within static/app/api.spec.tsx, replace the direct assignment to window.location with a call to the existing setWindowLocation() helper function to restore the URL. Alternatively, remove the restoration logic entirely.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.
Location: package.json#L224
Potential issue: The update to jest introduces JSDOM v26, which makes the
`window.location` property read-only and non-configurable. In `static/app/api.spec.tsx`,
the `afterEach` hook for the `resolveHostname` test suite attempts to restore the
original location by directly reassigning `window.location`. This will now throw a
`TypeError: Cannot set property location of #<Window> which has only a getter` at
runtime, causing all tests within that suite to fail during the cleanup phase.
Did we get this right? 👍 / 👎 to inform future reviews.
|
Looks like these dependencies are no longer updatable, so this is no longer needed. |
Bumps the jest-dependencies group with 2 updates: jest and jest-environment-jsdom.
Updates
jestfrom 30.0.4 to 30.2.0Release notes
Sourced from jest's releases.
Changelog
Sourced from jest's changelog.
... (truncated)
Commits
855864ev30.2.0da9b532v30.1.3ebfa31cv30.1.2d347c0fv30.1.14d5f41dv30.1.022236cfv30.0.5Updates
jest-environment-jsdomfrom 30.0.4 to 30.2.0Release notes
Sourced from jest-environment-jsdom's releases.
Changelog
Sourced from jest-environment-jsdom's changelog.
... (truncated)
Commits
855864ev30.2.0ebfa31cv30.1.2d347c0fv30.1.14d5f41dv30.1.022236cfv30.0.5You can trigger a rebase of this PR by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions